Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop support for MongoDB 4.4 #2617

Merged
merged 1 commit into from
Mar 27, 2024
Merged

Drop support for MongoDB 4.4 #2617

merged 1 commit into from
Mar 27, 2024

Conversation

malarzm
Copy link
Member

@malarzm malarzm commented Mar 19, 2024

MongoDB 4.4 reached its EOL in February so let's make the CI lighter for upcoming version

@malarzm malarzm added the Task label Mar 19, 2024
@malarzm malarzm added this to the 2.8.0 milestone Mar 19, 2024
@SenseException
Copy link
Member

SenseException commented Mar 20, 2024

If 4.4 gets dropped, does it make sense to have older versions mentioned in the docs? At this point multi-document transactions are a given.
https:/doctrine/mongodb-odm/blob/2.8.x/docs/en/reference/transactions-and-concurrency.rst?plain=1#L18-L19

@malarzm
Copy link
Member Author

malarzm commented Mar 21, 2024

Good catch! It does not. Github did not showed me those when grepping for 4.4 with UI. Will adjust it later :)

MongoDB 4.4 reached its EOL in February so let's make the CI lighter for upcoming version
@GromNaN
Copy link
Member

GromNaN commented Mar 26, 2024

For the record, even if the server is no longer supported in version 4.4, we continue to maintain the driver for 3 years from the server version EOL.

@malarzm
Copy link
Member Author

malarzm commented Mar 27, 2024

For the record, even if the server is no longer supported in version 4.4, we continue to maintain the driver for 3 years from the server version EOL.

Luckily enough no code paths were made, so technically 4.4 still works with ODM. We're just not testing against it which I think is fair given it has reached EOL :)

I'll be merging this PR as CI fails are not related.

@malarzm malarzm merged commit 26ae546 into 2.8.x Mar 27, 2024
28 of 34 checks passed
@malarzm malarzm deleted the malarzm-patch-1 branch March 27, 2024 20:24
@malarzm
Copy link
Member Author

malarzm commented Mar 27, 2024

Dang, I've missed some 5.0 sharded cluster fail. That's what one gets from a broken CI :( I'll be looking into it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants